Auto merge of #3570 - integer32llc:fix-slug-link, r=alexcrichton

Link to the category slugs page from the docs instead

Oops, just noticed that I didn't change this link in the docs when we changed to specifying slugs for categories in `Cargo.toml` :-/

Auto merge of #3569 - malept:manifest-example-typo-fix, r=alexcrichton

Fix typo in badges section of example manifest

Extremely small typo I saw when I was looking at the new badges feature.

Link to the category slugs page from the docs instead

Auto merge of #3556 - KalitaAlexey:allow-examples-to-be-library, r=alexcrichton

Allow examples to be library

This PR allows to specify **crate-type** to an example in **Cargo.toml**.
After this PR an example's **crate-type** can be:

* lib
* rlib
* dylib
* proc-macro

Please look at src/cargo/core/manifest.rs:116 because I am not sure whether I done it right.

I haven't added any tests.
I'd like to add them if someone says me how to do that.

Fixes #2358.

Fix typo in badges section of example manifest

Auto merge of #3554 - bmwiedemann:patch-1, r=alexcrichton

make build reproducible

see https://reproducible-builds.org/ for why this is good
and https://wiki.debian.org/ReproducibleBuilds/TimestampsProposal#Bash_.2F_POSIX_shell for variants that also work on BSD

Merge branch 'master' into allow-examples-to-be-library

Auto merge of #3568 - alexcrichton:fix-small, r=alexcrichton

Update travis token and fix warning

Update travis token and fix warning

make build reproducible

by allowing build systems to use a constant date instead of the current one
see https://reproducible-builds.org/ for why this is good

This call is designed to work with both GNU date and BSD date implementations

Fixed the test

Fixed tests and removed the unnecessary comment

Auto merge of #3565 - integer32llc:fix-gh-pages, r=alexcrichton

Fix doc publishing: Remove $USER from the --user argument from pip install

[This looks to be the cause of why docs aren't publishing to gh-pages](https://travis-ci.org/rust-lang/cargo/jobs/193507734#L1059). I was able to publish the docs to [my repo's gh-pages](https://integer32llc.github.io/cargo/) with this change (and setting my own secure gh-token).

It looks like this problem got introduced [in this commit](https://github.com/rust-lang/cargo/commit/dc15ca5d202d6d10f6be45f388759a902723de6c). I don't really know python/pip, but `$USER` wasn't there before and [Travis' docs just say `--user`](https://docs.travis-ci.com/user/languages/python/#pip).

Remove $USER from the --user argument from pip install

Fixed tests

Auto merge of #3563 - alexcrichton:upload-docs-again, r=alexcrichton

Run full Travis builds on all non-PR branches

This'll result in duplicate work when merging, but otherwise we're not uploading
docs.

Run full Travis builds on all non-PR branches

This'll result in duplicate work when merging, but otherwise we're not uploading
docs.

Auto merge of #3560 - KalitaAlexey:make-doc-subcommand-print-duration, r=alexcrichton

Made doc subcommand print duration

This PR makes `cargo doc` print duration.
Fixes #3016.

Made doc subcommand print duration

Auto merge of #3558 - matklad:fix-test, r=alexcrichton

Fix a test.

That's the test we've discussed in #3443

Added tests for building an example as a library

Fixed style issues

Fix a test.

Presumably, it should failed when it was first written, but Cargo does not do such validation yet.

Auto merge of #3552 - keeperofdakeys:proc-macro-doc-test, r=alexcrichton

Allow doc tests to run on proc macro crates

Fixes https://github.com/rust-lang/cargo/issues/3545

Since `--test` works for rustc, doctests should also work. Currently cargo isn't setup to run doctests for proc macro crates, this PR adds them to the list.

Currently rustdoc can run doctests for proc-macro crates, but the `phase_2_configure_and_expand` call` triggers the following warning:

```
the `#[proc_macro_derive]` attribute is only usable with crates of the `proc-macro` crate type
```

So perhaps this PR should wait until I've finished creating/testing the PR for rustc.

Auto merge of #3443 - matklad:path-dep-non-ws, r=alexcrichton

Path deps outside workspace are not members

closes #3192

This implements @Boscop suggestion: path dependencies pointing outside the workspace are never members.

Not sure that it handled #3192 fully: you can't exclude a path dependency within workspace. Not sure this is super useful though...

Add tests for doc testing proc macro crates

Auto merge of #3546 - integer32llc:badgers, r=alexcrichton

Upload Travis CI and Appveyor badge metadata specified in the manifest

This goes with rust-lang/crates.io#504. This has cargo upload badge metadata to crates.io on publish, and will print any warnings it gets back from crates.io about unknown badges or missing required badge attributes!

This will definitely cause some merge conflicts with #3301, I'll watch and fix whichever one gets merged 2nd :)

Upload badge metadata specified in the manifest

Merge branch 'master' of https://github.com/rust-lang/cargo into allow-examples-to-be-library

Allow examples to be libraries

Auto merge of #3301 - integer32llc:categories, r=alexcrichton

Upload categories specified in the manifest

This adds support for uploading categories to crates.io, if they are specified in the manifest.

This goes with rust-lang/crates.io#473. It should be fine to merge this PR either before or after that one; crates.io master doesn't care if the categories are in the metadata or not. With that PR, I was able to use this patch with cargo to add categories to a crate!

Allow doc tests to run on proc macro crates

Auto merge of #3542 - creativcoder:check-digit-name, r=alexcrichton

Add a check for names starting with a digit

According to Rust grammer https://doc.rust-lang.org/grammar.html#extern-crate-declarations for extern crate declarations, a crate name cannot start with a digit.

But, currently this rule is not upheld by `cargo new` as creating a project like:

`cargo new 2048` would create a project named 2048 which obviously won't compile with crate declaration like `extern crate 2048` by a consumer.

This obviously is a rare case in practice, but its always good to check i guess.

This PR adds a check to the `new` method and `bail`s out with a message for any names starting with a digit.

PS: I noticed it while making a 2048 puzzle game as a library so thought it would be nice to add this check :)

Add a check for names starting with a digit

More test for workspaces \w path dependencies

Document new rules about path deps in workspace

Path deps outside workspace are not members

closes #3192

Auto merge of #3534 - sfackler:opt-level, r=alexcrichton

Allow debuginfo level to be specified

`true` is mapped to `2`, which matches current behavior.

r? @alexcrichton

Allow debuginfo level to be specified

`true` is mapped to `2`, which matches current behavior.

Auto merge of #3533 - jtgeibel:3519-dev-target, r=alexcrichton

Rename debug target to dev in build output

This renames the debug target to dev in the build output for non-release
builds.  This includes the output of commands such as `cargo build`,
`cargo test`, and `cargo doc`.

Closes #3519

Auto merge of #3532 - alexcrichton:ignore, r=brson

Ignore spuriously failing Windows test

I really have no clue why this test is failing on Windows, and after months of
being unable to diagnose I'm tired of retrying PRs due to this failure. Let's
just ignore it on Windows.

Closes #3466

Auto merge of #3490 - alexcrichton:dev-dep-doc-test, r=brson

Fix cargo test --doc with dev-deps

Previously Cargo accidentally didn't pull in dev-dependencies due to the way
`cargo test --doc` was interpreted in terms of top-level targets. This PR
special cases this situation by ensuring that the doctest intention makes its
way all to the backend and the dependencies can be correctly calculated.

Closes #3422

Rename debug target to dev in build output

This renames the debug target to dev in the build output for non-release
builds.  This includes the output of commands such as `cargo build`,
`cargo test`, and `cargo doc`.

Closes #3519

Auto merge of #3478 - alexcrichton:lift, r=brson

Lift up workspace rlibs while building

I think the condition here was slightly off from before, so invert it subtly to
get what we want, lifting up anything in a workspace or binaries otherwise.

Closes #3432

Ignore spuriously failing Windows test

I really have no clue why this test is failing on Windows, and after months of
being unable to diagnose I'm tired of retrying PRs due to this failure. Let's
just ignore it on Windows.

Closes #3466

Auto merge of #3527 - nikomatsakis:master, r=alexcrichton

check for `CARGO_INCREMENTAL` and pass `-Zincremental` if present

Per the discussion on IRC, this adds a very simple way for cargo users to opt into incremental compilation by setting the `CARGO_INCREMENTAL` environment variable (i.e., `CARGO_INCREMENTAL=1 cargo build`). This will result in incremental data being stored into the `target/incremental` directory. Since it supplies `-Z`, this option is only intended for use on nightly compilers, though cargo makes no effort to check.

The plan is to keep incremental compilation optional until we are feeling more confident it's not going to cause problems for people. At that point, it should become part of the compilation profile. It will be the default when building in debug builds, and opt-in for release builds.

address suggestions from acrichto

check `cargo test` too`

check for `CARGO_INCREMENTAL` and pass `-Zincremental` if present

Auto merge of #3524 - sdroege:test-build-doc-test-check-status, r=alexcrichton

Check the exit code of all processes started in the build/test tests

By not checking the exit codes, a failure in the build::crate_env_vars()
test was ignored.

As suggested by @alexcrichton in https://github.com/rust-lang/cargo/pull/3515#discussion_r95253461

Check the exit code of all processes started in the build/test tests

By not checking the exit codes, a failure in the build::crate_env_vars()
test was ignored.

Auto merge of #3489 - matklad:rel-ws, r=alexcrichton

Use canonical paths for checking equality

Auto merge of #3515 - sdroege:doc-all, r=alexcrichton

 Add support for documenting all members of the workspace with "doc --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3518 - Manishearth:static-mut, r=alexcrichton

Replace static mut bool with atomicbool

unsafe in tests is fine, but it can be trivially avoided in this case.

r? @alexcrichton

cc @froydnj

Replace static mut bool with atomicbool

Collect library/binary names when documenting even if a spec was given

This seems more consistent and generally should work better.

Check if the "doc --all" tests exit with exit code 0

Auto merge of #3511 - sdroege:build-all, r=alexcrichton

Add support for building all members of the workspace with "build --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3510 - froydnj:cross-compilation-test-checking, r=alexcrichton

handle cross-compilation test failure more gracefully

The current cross-compilation test module fails every test if an appropriate cross standard library is not found.  This behavior is unhelpful, because the failures are verbose, unexpected, and there's no
obvious way to make them go away.

Instead, it would be better to check once before all the tests that an appropriate cross-compilation setup is available.  Once that check has been done, a single test can fail with an appropriate `panic!` message while the other tests silently pass.  The `panic!` message can inform the user about their options, either setting an appropriate environment variable, or using rustup to install the necessary cross standard library.

Assuming the user has rustup installed, the single failure now looks
something like:

```
thread 'plugin_deps' panicked at 'Cannot cross compile to i686-unknown-linux-gnu.

This failure can be safely ignored. If you would prefer to not see this
failure, you can set the environment variable CFG_DISABLE_CROSS_TESTS to "1".

Alternatively, you can install the necessary libraries for cross-compilation with

    rustup toolchain install stable-i686-unknown-linux-gnu
', tests/cross-compile.rs:87
note: Run with `RUST_BACKTRACE=1` for a backtrace.
```

The code is admittedly gnarly, with synchronization and `unsafe` sections, but I wasn't sure how to make it any better. Suggestions welcome!

Fixes #3086.

Add tests for "build --all"

These are basically the same as the ones from "test --all"

Add tests for "doc --all"

These are basically the same as the ones from "test --all" and "build --all"

Add support for documenting all members of the workspace with "doc --all"

https://github.com/rust-lang/cargo/issues/3491

Normalize paths before checking for equality

handle cross-compilation test failure more gracefully

The current cross-compilation test module fails every test if an
appropriate cross standard library is not found.  This behavior is
unhelpful, because the failures are verbose, unexpected, and there's no
obvious way to make them go away.

Instead, it would be better to check once before all the tests that an
appropriate cross-compilation setup is available.  Once that check has
been done, a single test can fail with an appropriate `panic!` message
while the other tests silently pass.  The `panic!` message can inform
the user about their options, either setting an appropriate environment
variable, or using rustup to install the necessary cross standard
library.

Assuming the user has rustup installed, the single failure now looks
something like:

```
thread 'plugin_deps' panicked at 'Cannot cross compile to i686-unknown-linux-gnu.

This failure can be safely ignored. If you would prefer to not see this
failure, you can set the environment variable CFG_DISABLE_CROSS_TESTS to "1".

Alternatively, you can install the necessary libraries for cross-compilation with

    rustup toolchain install stable-i686-unknown-linux-gnu

You may need to install runtime libraries for your Linux distribution as well.
', tests/cross-compile.rs:87
note: Run with `RUST_BACKTRACE=1` for a backtrace.
```

Fixes #3086.

Auto merge of #3509 - froydnj:colored-shell-tests, r=alexcrichton

disable color shell tests for terminals that don't support color

Some terminals (e.g. running a shell inside Emacs's shell-mode) don't
support color, and running tests that assume the terminal supports color
don't work so well.  Instead, if color is expected, check the terminal
for whether it supports color or not, and act accordingly.

Add support for building all members of the workspace with "build --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3508 - petrochenkov:bsover, r=alexcrichton

Do not treat `warning` and `rerun-if-changed` as user-defined metadata in build script overrides

Fixes https://github.com/rust-lang/cargo/pull/3438#discussion_r94018355
r? @alexcrichton

disable color shell tests for terminals that don't support color

Some terminals (e.g. running a shell inside Emacs's shell-mode) don't
support color, and running tests that assume the terminal supports color
don't work so well.  Instead, if color is expected, check the terminal
for whether it supports color or not, and act accordingly.

Auto merge of #3504 - froydnj:win32-dependencies, r=alexcrichton

make certain dependencies Windows-only

Inspired by servo/heapsize#71, we make certain dependencies only required under Windows.

Do not treat `warning` and `rerun-if-changed` as user-defined metadata in build script overrides

make certain dependencies Windows-only

Inspired by servo/heapsize#71, we make certain dependencies only
required under Windows.

Auto merge of #3427 - nrc:api-check-2, r=alexcrichton

Make some aspects of check/build available as an API.

There are two key parts to this commit:
* let API clients run `cargo check` with minimal fuss (ops/cargo_check.rs),
* let API clients intercept and customise Cargo's calls to rustc (all the Executor stuff).

r? @alexcrichton

Auto merge of #3505 - froydnj:read2-delete-try, r=alexcrichton

delete unused try macro from read2.rs

AFAICS, this is unused.

Auto merge of #3507 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Remove extraneous .BaseName from AppVeyor

Apparently this isn't necessary

Remove extraneous .BaseName from AppVeyor

Apparently this isn't necessary

Add `--profile check` to `cargo rustc`

delete unused try macro from read2.rs

Auto merge of #3495 - sfackler:bump-shell-escape, r=alexcrichton

Update shell-escape

r? @alexcrichton

Auto merge of #3500 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Try using before_deploy on AppVeyor

Try using before_deploy on AppVeyor

Fix cargo test --doc with dev-deps

Previously Cargo accidentally didn't pull in dev-dependencies due to the way
`cargo test --doc` was interpreted in terms of top-level targets. This PR
special cases this situation by ensuring that the doctest intention makes its
way all to the backend and the dependencies can be correctly calculated.

Closes #3422

Lift up workspace rlibs while building

I think the condition here was slightly off from before, so invert it subtly to
get what we want, lifting up anything in a workspace or binaries otherwise.

Closes #3432

Update shell-escape

review changes, round 2 part b

Remove the ContinueBuild concept, be lenient about missing files in all cases, and undo the fingerprint changes (not an issue now we're not stopping the build early).

review changes, round 2

Mostly focussing on the ergonomics of the API - removes with_* methods with closures, and replaces generics with trait objects.

Review changes

Make some aspects of check/build available as an API.

There are two key parts to this commit:
* let API clients run `cargo check` with minimal fuss (ops/cargo_check.rs),
* let API clients intercept and customise Cargo's calls to rustc (all the Executor stuff).

Auto merge of #3468 - nrc:metadata-emit, r=alexcrichton

cargo check: use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571 (don't land before that does)

r? @alexcrichton

Fixup tests

Use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571

Auto merge of #3485 - sfackler:useragent, r=alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

r? @alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

Auto merge of #3480 - alexcrichton:one-flaky-test, r=brson

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Auto merge of #3477 - sfackler:shell-escape, r=alexcrichton

Use externally sourced shell-escape

Closes #3374

r? @alexcrichton

Use externally sourced shell-escape

Closes #3374

Auto merge of #3470 - alexcrichton:no-more-urls, r=steveklabnik

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Auto merge of #3435 - alexcrichton:bump-libssh2, r=brson

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Auto merge of #3417 - alexcrichton:fix-svn, r=brson

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414